update Inter font to version 4.1 (fix bedrock #15999)#1026
update Inter font to version 4.1 (fix bedrock #15999)#1026stephaniehobson merged 3 commits intomozilla:mainfrom
Conversation
wen-2018
force-pushed
the
font-inter-new-version
branch
from
25cc364 to
609db6e
Compare
|
Some previous exploration of variable fonts was done in 2022: #810 We might want to look at if 3 or more Inter fonts are loaded on the new refresh designed pages to get a more current performance comparison edit: in-product pages like WNP always use Inter, so probably best to check font needs there first |
wen-2018
force-pushed
the
font-inter-new-version
branch
from
609db6e to
fc3f571
Compare
wen-2018
changed the title
wen-2018
force-pushed
the
font-inter-new-version
branch
from
fc3f571 to
e514b8c
Compare
maureenlholland
left a comment
maureenlholland
left a comment
There was a problem hiding this comment.
LGTM r+wc ✍️
suggestion (non-blocking) update CHANGELOG to include the info about font version upgrade
seems a bit odd to me that the prod file sizes are smaller than the PR, but it's not a huge difference
PROD
PR
maureenlholland
left a comment
maureenlholland
left a comment
There was a problem hiding this comment.
minor requested change on the changelog, then good to go
wen-2018
force-pushed
the
font-inter-new-version
branch
from
0158e04 to
4a2f22c
Compare
maureenlholland
left a comment
maureenlholland
left a comment
There was a problem hiding this comment.
r+wc
after latest procotol release, changelog needs a minor update
wen-2018
force-pushed
the
font-inter-new-version
branch
from
4a2f22c to
dacaed3
Compare
4 participants
Description
This PR updates the Inter font to version 4.1
CHANGELOG.md.Issue
mozilla/bedrock#15999
Testing
Enter helpful notes for whoever code reviews this change.